오셨군요! 내용 확인해보겠습니다 ~

리뷰 진행상황

• replacement of feat: add Google Vertex AI connector implementation #454
• 지난 PR에서 논의/확인했던 사항
  • GeminiChatClient 사용해서 Connector 구현
  • 테스트 시나리오 커버리지
    • GoogleVertexAIConnector가 LanguageModelConnector를 상속하는지에 대한 테스트
    • Connector 로직 테스트
      • EnsureLanguageModelSettingsValid()
      • GetChatClientAsync()
      • LanguageModelConnector.CreateChatClientAsync()
      • 예외발생 테스트 컨벤션 Tests are NOT testing exceptions #451
      • 하드코딩된 magic string 상수화
    • 경계값 테스트
    • LanguageModelConnector.cs case문에서 LLM Provider 지원순서를 지키기

리뷰 백로그 / 기타 메모

• 그래서 이 커넥터는 잘 동작하는가 ...
• 문서 : 호스트머신, 컨테이너 부분
• Bicep 구현
• 문서 : Bicep/Azure 부분

test/OpenChat.PlaygroundApp.Tests/Connectors/GoogleVertexAIConnectorTests.cs 테스트 시나리오 정리

• Given_Settings_Is_Null_When_EnsureLanguageModelSettingsValid_Invoked_Then_It_Should_Thro
  • GoogleVertexAI 설정이 null일 때 EnsureLanguageModelSettingsValid 호출 시 예외가 발생해야 함
• Given_Invalid_ApiKey_When_EnsureLanguageModelSettingsValid_Invoked_Then_It_Should_Thro
  • ApiKey가 null, 빈 문자열, 공백만 있을 때 EnsureLanguageModelSettingsValid 호출 시 예외가 발생해야 함
• Given_Invalid_Model_When_EnsureLanguageModelSettingsValid_Invoked_Then_It_Should_Thro
  • Model이 null, 빈 문자열, 공백만 있을 때 EnsureLanguageModelSettingsValid 호출 시 예외가 발생해야 함
• Given_Valid_Settings_When_EnsureLanguageModelSettingsValid_Invoked_Then_It_Should_Return_Tru
  • 올바른 설정이 주어졌을 때 EnsureLanguageModelSettingsValid 호출 시 true를 반환해야 함
• Given_Valid_Settings_When_GetChatClient_Invoked_Then_It_Should_Return_ChatClien
  • 올바른 설정이 주어졌을 때 GetChatClientAsync 호출 시 ChatClient가 반환되어야 함
• Given_Settings_Is_Null_When_GetChatClientAsync_Invoked_Then_It_Should_Thro
  • GoogleVertexAI 설정이 null일 때 GetChatClientAsync 호출 시 예외가 발생해야 함
• Given_Missing_ApiKey_When_GetChatClient_Invoked_Then_It_Should_Thro
  • ApiKey가 null 또는 빈 문자열일 때 GetChatClientAsync 호출 시 예외가 발생해야 함
• Given_Missing_Model_When_GetChatClient_Invoked_Then_It_Should_Thro
  • Model이 null일 때 GetChatClientAsync 호출 시 예외가 발생해야 함
• Given_Valid_Settings_When_CreateChatClientAsync_Invoked_Then_It_Should_Return_ChatClien
  • 올바른 설정이 주어졌을 때 LanguageModelConnector.CreateChatClientAsync 호출 시 IChatClient가 반환되어야 함
• Given_Invalid_Settings_When_CreateChatClientAsync_Invoked_Then_It_Should_Thro
  • 잘못된 설정(예: ApiKey가 null)이 주어졌을 때 LanguageModelConnector.CreateChatClientAsync 호출 시 예외가 발생해야 함

피드백 주신 부분 하나씩 수정하면서 코멘트 남기겠습니다!

우선 PR 본문 정리했습니다!

test/OpenChat.PlaygroundApp.Tests/Abstractions/LanguageModelConnectorTests.cs 부분 주석 해제 했습니다!

@hxcva1 #494 PR이 머지가 됐으므로 테스트 코드는 이를 참고해서 작성해 주세욥! 촘촘하게 작성하셔야 합니다.

@hxcva1 잘 지내시죠? 진행상황 업데이트 부탁드립니다
기한이 있어서 필요하면 저도 같이 작업 진행해볼게요

그러면 저는 우선 테스트 코드 보완 부터 시도해보도록 하겠습니다

Github 상에서 Conflict 해결하고 커밋을 했는데 로컬에서 이 버전을 pull 안하고 작업한 내용을 커밋했는데 푸쉬하려고 하니 에러가 나서 어떻게 해야할까요? rebase 기능을 사용해도 되는지 git이 아직 완전히 익숙하지는 않아서 죄송합니다.

Github 상에서 Conflict 해결하고 커밋을 했는데 로컬에서 이 버전을 pull 안하고 작업한 내용을 커밋했는데 푸쉬하려고 하니 에러가 나서 어떻게 해야할까요? rebase 기능을 사용해도 되는지 git이 아직 완전히 익숙하지는 않아서 죄송합니다.

git pull 해보셨나요?
git fetch & git rebase origin/<branch명> 하셔도 되겠지만, 그냥 git pull 하셔도 됩니다.

conflict 발생하면 로컬에서 수정하고 commit & push 하시면 됩니다.

@hxcva1 CI 터지는 테스트 고쳐주세요

• 테스트 방법

  dotnet test test/OpenChat.PlaygroundApp.Tests/OpenChat.PlaygroundApp.Tests.csproj --filter "FullyQualifiedName~Ollama"

• 테스트 리뷰 체크리스트
  • LanguageModelConnector를 상속하는지에 대한 테스트
  • 예외발생 테스트 컨벤션 Tests are NOT testing exceptions #451
  • 하드코딩된 magic string 상수화
  • 경계값 테스트
  • 테스트 시나리오 커버리지
  • Connector 로직 테스트
    • 생성자 : 각 파라미터별 실패, 전체 Valid and 성공
    • EnsureLanguageModelSettingsValid() : 각 파라미터별 실패, 전체 Valid and 성공
    • GetChatClientAsync() : 각 파라미터별 실패, 전체 Valid and 성공
    • LanguageModelConnector.CreateChatClientAsync() : 각 파라미터별 실패, 전체 Valid and 성공

• Azure 배포 테스트중
• Azure 배포후 컨테이너에서 API_KEY 이슈. API_KEY 길이가 유효하지 않다고 나오는데, 값이 유효하지 않거나 공백으로 전달. 컨테이너 내부에 오류가 발생하니 반복하여 새로운 컨테이너가 생성되는 증상.

완료! secretRef 누락으로 컨테이너에 값 전달 안되던 상황.